Skip to content

Hid some PerformanceWarnings...#596

Merged
knirirr merged 2 commits into
developfrom
isatools_materials_test_deprecation
Oct 31, 2025
Merged

Hid some PerformanceWarnings...#596
knirirr merged 2 commits into
developfrom
isatools_materials_test_deprecation

Conversation

@knirirr

@knirirr knirirr commented Oct 29, 2025

Copy link
Copy Markdown
Contributor

There are still some warnings remaining.
Some have been hidden, for Pandas, and a link to an explanation has been provided.

@knirirr

knirirr commented Oct 29, 2025

Copy link
Copy Markdown
Contributor Author

The first run of this triggered:

tests/test_clients/test_mw2isa.py:38: AssertionError
----------------------------- Captured stdout call -----------------------------
user options {'studyid': 'ST000367', 'outputdir': '/tmp/tmpdc446jk1', 'dl_option': 'no', 'validate_option': True}
Error: in main() method something went wrong
conversion failed

...which appears to be caused by the tests calling a live URL (http://www.metabolomicsworkbench.org/...) on line 1070 of isatools/net/mw2isa/init.py, which timed out.

@proccaserra

Copy link
Copy Markdown
Member

This should indeed be mocked to avoid such situations. I'll open a ticket.

@knirirr knirirr requested a review from proccaserra October 30, 2025 14:08
@coveralls

Copy link
Copy Markdown

Coverage Status

coverage: 82.314% (+0.004%) from 82.31%
when pulling 8976a8e on isatools_materials_test_deprecation
into 68ed2c4 on develop.

@proccaserra proccaserra left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

looks good to me. opening an issue for the OLS service as discussed

@knirirr knirirr merged commit cbd152a into develop Oct 31, 2025
12 checks passed
@knirirr knirirr deleted the isatools_materials_test_deprecation branch October 31, 2025 10:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants